Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[scripts/report_failed_tests] fix report_failed_tests integration on CI #71131

Merged
merged 8 commits into from
Jul 13, 2020

Conversation

spalger
Copy link
Contributor

@spalger spalger commented Jul 8, 2020

We moved the location of junit files in #64011 and didn't update the paths that scripts/report_failed_tests looks for junit in. This PR updates the script to throw when no junit is found, and modifies the script to look for files as configured via the CLI.

@spalger spalger added Team:Operations Team label for Operations Team v8.0.0 release_note:skip Skip the PR/issue when compiling release notes v7.9.0 labels Jul 8, 2020
@spalger
Copy link
Contributor Author

spalger commented Jul 8, 2020

@brianseeders you'll likely want to apply the inverse of 0c101f5 (#71131) when you resubmit the tasks PR

@spalger spalger marked this pull request as ready for review July 9, 2020 18:40
@spalger spalger requested a review from a team as a code owner July 9, 2020 18:40
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-operations (Team:Operations)

@spalger
Copy link
Contributor Author

spalger commented Jul 9, 2020

@elasticmachine merge upstream

@spalger
Copy link
Contributor Author

spalger commented Jul 13, 2020

@elasticmachine merge upstream

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Build metrics

✅ unchanged

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@spalger spalger merged commit ec43d45 into elastic:master Jul 13, 2020
@spalger spalger deleted the fix/report-failed-tests branch July 13, 2020 19:29
@spalger spalger added the v7.8.1 label Jul 13, 2020
spalger added a commit to spalger/kibana that referenced this pull request Jul 13, 2020
…CI (elastic#71131)

Co-authored-by: spalger <spalger@users.noreply.github.com>
Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
spalger added a commit to spalger/kibana that referenced this pull request Jul 13, 2020
…CI (elastic#71131)

Co-authored-by: spalger <spalger@users.noreply.github.com>
Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
spalger added a commit that referenced this pull request Jul 13, 2020
…on on CI (#71131) (#71524)

Co-authored-by: spalger <spalger@users.noreply.github.com>
Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
spalger added a commit that referenced this pull request Jul 13, 2020
…on on CI (#71131) (#71525)

Co-authored-by: spalger <spalger@users.noreply.github.com>
Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>

Co-authored-by: spalger <spalger@users.noreply.github.com>
Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release_note:skip Skip the PR/issue when compiling release notes Team:Operations Team label for Operations Team v7.8.1 v7.9.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants